Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Workplace Search] Fix reauthenticate links for OneDrive and SharePoint #96271

Merged
merged 1 commit into from
Apr 6, 2021

Conversation

jgr
Copy link

@jgr jgr commented Apr 6, 2021

Summary

For both OneDrive and SharePoint we define a service_type that has an
underscore in the middle (one_drive and share_point). This fixes the
route definitions so sources of these connector types can be
reauthenticated.

Without these changes, the "Fix" link for these sources 404's.

related to https://github.com/elastic/workplace-search-team/issues/1572

For both OneDrive and SharePoint we define a service_type that has an
underscore in the middle (one_drive and share_point). This fixes the
route definitions so sources of these connector types can be
reauthenticated.
@jgr jgr added Feature:Plugins release_note:skip Skip the PR/issue when compiling release notes Team:WorkplaceSearch v7.13.0 auto-backport Deprecated - use backport:version if exact versions are needed labels Apr 6, 2021
@jgr jgr requested a review from a team April 6, 2021 00:10
Copy link
Contributor

@scottybollinger scottybollinger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks so much for knocking this out James!

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
enterpriseSearch 1.9MB 1.9MB +4.0B

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@jgr jgr merged commit 4610764 into elastic:master Apr 6, 2021
@jgr jgr deleted the jgr/fix-microsoft-reauth-routes branch April 6, 2021 19:19
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Apr 6, 2021
For both OneDrive and SharePoint we define a service_type that has an
underscore in the middle (one_drive and share_point). This fixes the
route definitions so sources of these connector types can be
reauthenticated.
@kibanamachine
Copy link
Contributor

💚 Backport successful

7.x / #96347

This backport PR will be merged automatically after passing CI.

kibanamachine added a commit that referenced this pull request Apr 6, 2021
For both OneDrive and SharePoint we define a service_type that has an
underscore in the middle (one_drive and share_point). This fixes the
route definitions so sources of these connector types can be
reauthenticated.

Co-authored-by: James Rucker <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed Feature:Plugins release_note:skip Skip the PR/issue when compiling release notes Team:EnterpriseSearch v7.13.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants